Skip to content

Conversation

treblereel
Copy link
Collaborator

@treblereel treblereel commented Sep 23, 2025

[Many thanks for submitting your Pull Request ❤️!

What this PR does / why we need it:
Fix #477

Special notes for reviewers:

Additional information (if needed):]

@treblereel treblereel marked this pull request as draft September 23, 2025 17:12
@treblereel
Copy link
Collaborator Author

I'm gonna add more tests soon

@treblereel treblereel marked this pull request as ready for review October 9, 2025 23:57
Signed-off-by: Dmitrii Tikhomirov <chani.liet@gmail.com>
Signed-off-by: Dmitrii Tikhomirov <chani.liet@gmail.com>
Signed-off-by: Dmitrii Tikhomirov <chani.liet@gmail.com>
Signed-off-by: Dmitrii Tikhomirov <chani.liet@gmail.com>
Signed-off-by: Dmitrii Tikhomirov <chani.liet@gmail.com>
Comment on lines +75 to +77
} catch (Exception e) {
throw new RuntimeException(e);
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You do not need this block, you just want to propagate all exception that are not workflow (because if it workflow you want to try the next server), which will happen anyway if you delete this block

improve ExpressionURISupplier

Signed-off-by: Dmitrii Tikhomirov <chani.liet@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add OpenAPI task implementation

2 participants